-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump ruby 3.3 #3697
base: main
Are you sure you want to change the base?
Bump ruby 3.3 #3697
Conversation
380556b
to
ca59d73
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.rubocop.yml is missing an update to the TargetRubyVersion
My apologies for jumping the gun, I had mistaken this with another PR in the dev-pull-request channel and realise now that this might not be ready for review. |
I thought this was ready too, but library-cms is a higher priority so we'll do that first. I appreciate any feedback you have! |
csv was loaded from the standard library, but is not part of the default gems starting from Ruby 3.4.0. You can add csv to your Gemfile or gemspec to silence this warning.
ostruct.rb was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
More easily available in RHEL/centos ecosystem. Can revisit 3.4 when it's more mature.
c7196d5
to
8fedf6d
Compare
Layout/LineLength: Line is too long. [153/120] (https://rubystyle.guide#max-line-length)
8fedf6d
to
ec402c9
Compare
Context
This morning breakman warned us that
Related to https://github.com/ualbertalib/library_applications_development/issues/163
What's New
Bump Ruby to 3.3 in Gemfile, working dev environment, and part of CI strategy (include 3.4 as well).
Not done: docker environment